src/main/java/balancetalk/talkpick/application/TalkPickFileService.java (1)

비동기 메서드에서 매개변수 리스트의 안전한 복사본 생성 필요

handleFilesOnTalkPickUpdate 메서드가 @Async로 비동기 실행되므로, 매개변수로 전달된 리스트의 직접적인 수정은 동시성 문제를 일으킬 수 있습니다. 다음과 같이 수정하는 것이 안전합니다:

public void handleFilesOnTalkPickUpdate(List<Long> newFileIds, List<Long> deleteFileIds, Long talkPickId) {
    deleteFiles(deleteFileIds);
    List<Long> safeNewFileIds = new ArrayList<>(newFileIds);
    safeNewFileIds.removeIf(deleteFileIds::contains);
    relocateFiles(safeNewFileIds, talkPickId);
}

🔗 Analysis chain

Line range hint 41-45: 업데이트 메서드의 동시성 이슈 가능성 검토 필요

newFileIds.removeIf((deleteFileIds::contains)) 연산이 동시성 문제를 일으킬 수 있습니다. 리스트 수정 전에 복사본을 만드는 것이 안전할 수 있습니다.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 동시성 관련 이슈가 있는지 코드베이스 검사
rg -l "handleFilesOnTalkPickUpdate.*@Async" | xargs rg "removeIf|remove|clear|add" -A 5 -B 5

Length of output: 44611

src/main/java/balancetalk/talkpick/domain/repository/TalkPickRepository.java (1)

14-14: 비동기 처리를 위한 메서드 시그니처가 적절합니다.

findAllBySummaryStatus 메서드는:

• Spring Data JPA의 명명 규칙을 잘 따르고 있습니다
• 전체 엔티티를 반환하여 N+1 문제를 방지할 수 있습니다
• 이벤트 기반 처리를 위한 데이터 조회에 적합합니다

하나 제안드리고 싶은 점은, 대량의 데이터 처리가 예상된다면 페이징 처리를 고려해보시는 것도 좋을 것 같습니다.

src/main/java/balancetalk/talkpick/application/TalkPickScheduleService.java (1)

16-16: 실패한 요약 재시도 로직이 효과적으로 개선되었습니다.

기존의 복잡한 구현이 단일 서비스 메서드 호출로 단순화되어 다음과 같은 이점이 있습니다:

• 코드의 가독성 향상
• 관심사의 명확한 분리
• 불필요한 비동기 처리 제거로 리소스 사용 최적화

src/main/java/balancetalk/talkpick/application/TalkPickService.java (1)

57-58: 파일 조회 시 데이터베이스 호출 최적화 제안

fileRepository를 사용하여 이미지 URL과 파일 ID를 각각 조회하고 있는데, 이는 두 번의 데이터베이스 호출을 발생시킬 수 있습니다. 한 번의 쿼리로 필요한 정보를 모두 가져올 수 있도록 리포지토리 메서드를 개선하는 것을 고려해보시기 바랍니다.

src/main/java/balancetalk/talkpick/domain/event/TalkPickEventHandler.java (1)

22-27: 업데이트 이벤트에서 불필요한 요약 작업 검토 필요

파일만 변경되고 내용이 변경되지 않은 경우에도 요약 작업이 수행됩니다. 내용이 변경된 경우에만 요약 작업을 수행하도록 최적화가 필요합니다.

src/main/java/balancetalk/talkpick/application/TalkPickFileService.java (1)

Line range hint 55-65: 존재 여부 확인 로직 최적화 필요

notExistsFilesBy 메서드를 호출한 후 다시 파일을 조회하는 것은 비효율적일 수 있습니다. 한 번의 쿼리로 처리하는 것이 좋겠습니다.

    @Async
    @Retryable(backoff = @Backoff(delay = 1000))
    @Transactional
    public void handleFilesOnTalkPickDelete(Long talkPickId) {
-       if (notExistsFilesBy(talkPickId)) {
-           return;
-       }
        List<File> files = fileRepository.findAllByResourceIdAndFileType(talkPickId, TALK_PICK);
+       if (files.isEmpty()) {
+           return;
+       }
        fileHandler.deleteFiles(files);
    }
-
-    private boolean notExistsFilesBy(Long talkPickId) {
-        return !fileRepository.existsByResourceIdAndFileType(talkPickId, TALK_PICK);
-    }

src/test/java/balancetalk/talkpick/application/TalkPickServiceTest.java (2)

71-73: 테스트 데이터 설정이 개선되어야 합니다.

빈 리스트 대신 실제와 유사한 테스트 데이터를 사용하여 테스트하는 것이 좋겠습니다.

-       when(fileRepository.findImgUrlsByResourceIdAndFileType(1L, TALK_PICK)).thenReturn(List.of());
-       when(fileRepository.findIdsByResourceIdAndFileType(1L, TALK_PICK)).thenReturn(List.of());
+       when(fileRepository.findImgUrlsByResourceIdAndFileType(1L, TALK_PICK))
+           .thenReturn(List.of("image1.jpg", "image2.jpg"));
+       when(fileRepository.findIdsByResourceIdAndFileType(1L, TALK_PICK))
+           .thenReturn(List.of(1L, 2L));

---

Line range hint 87-105: 테스트 케이스 중복 코드 제거 필요

여러 테스트에서 반복되는 given 부분을 @beforeeach로 이동하여 코드 중복을 제거하는 것이 좋겠습니다.

    @BeforeEach
    void setUp() {
        // ... 기존 코드 ...
+       when(talkPickRepository.findById(1L)).thenReturn(Optional.ofNullable(talkPick));
+       when(guestOrApiMember.isGuest()).thenReturn(true);
+       when(fileRepository.findImgUrlsByResourceIdAndFileType(1L, TALK_PICK))
+           .thenReturn(List.of("image1.jpg"));
+       when(fileRepository.findIdsByResourceIdAndFileType(1L, TALK_PICK))
+           .thenReturn(List.of(1L));
    }

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

src/test/java/balancetalk/talkpick/application/TalkPickServiceTest.java (1)

71-72: 테스트 메서드 변경이 일관성 있게 이루어졌습니다!

파일 저장소 메서드 호출로의 변경이 모든 테스트 케이스에 올바르게 적용되었습니다.

테스트 유지보수성 개선을 위한 제안

여러 테스트 메서드에서 동일한 mock 설정이 반복되고 있습니다. 이를 @BeforeEach에서 처리하는 것이 좋을 것 같습니다.

다음과 같이 수정하는 것을 고려해보세요:

 @BeforeEach
 void setUp() {
     talkPick = mock(TalkPick.class);
     member = mock(Member.class);
     guestOrApiMember = mock(GuestOrApiMember.class);

     when(talkPick.getId()).thenReturn(1L);
     // ... existing setup ...
+    when(fileRepository.findImgUrlsByResourceIdAndFileType(1L, TALK_PICK)).thenReturn(List.of());
+    when(fileRepository.findIdsByResourceIdAndFileType(1L, TALK_PICK)).thenReturn(List.of());
 }

Also applies to: 87-88, 103-104

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro